-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move "easy" basic checks scripts to the framework #95
Move "easy" basic checks scripts to the framework #95
Conversation
Signed-off-by: Nadav Tasher <[email protected]>
Improve supply chain security by including a SBOM file with substituted values. This will be used to construct a composite platform SBOM. Signed-off-by: Richard Hughes <[email protected]>
This commit removes irrelevant components that are no longer needed in the 4.0 release. Signed-off-by: Harry Ramsey <[email protected]>
ba96d78
to
31423be
Compare
Looks good, however given this #80 (comment) might need to move files from |
fb36949
to
8b06ebb
Compare
This commit fixes a condition where the incorrect submodule error message was displayed as we were using the CMAKE_CURRENT_SOURCE_DIR variable instead of MBEDTLS_DIR. Signed-off-by: Harry Ramsey <[email protected]>
This commit fixes a condition where the incorrect submodule error message was displayed as the check for the directory returns true only if the directory does not exist Signed-off-by: Harry Ramsey <[email protected]>
279f311
to
0fed0d6
Compare
scripts/recursion.pl
Outdated
# When the recursion depth might depend on data controlled by the attacker in | ||
# an unbounded way, those functions should use iteration instead. | ||
# | ||
# Typical usage: scripts/recursion.pl library/*.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't this be
# Typical usage: scripts/recursion.pl library/*.c | |
# Typical usage: framework/scripts/recursion.pl library/*.c |
now that it's moved?
0fed0d6
to
6c7796f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
MBED_TLS_USE_PSA_CRYPTO is now always enabled we need to remove documentation discussing cases when it is disabled. Signed-off-by: Janos Follath <[email protected]>
MBED_TLS_USE_PSA_CRYPTO is now always enabled we need to remove documentation discussing cases when it is disabled. Signed-off-by: Janos Follath <[email protected]>
This document is describes the testing strategy for the `MBEDTLS_USE_PSA_CRYPTO` option. This option is now always on, can't be disabled and the corresponding behaviour is the only library behaviour. Signed-off-by: Janos Follath <[email protected]>
MBED_TLS_USE_PSA_CRYPTO is now always enabled we need to remove documentation discussing cases when it is disabled. The goal is not to update the document, only to remove MBED_TLS_USE_PSA_CRYPTO, while making a minimal local context of the occurrance up to date and sensible. Signed-off-by: Janos Follath <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
Move check_files.py to the framework
Signed-off-by: Elena Uziunaite <[email protected]>
Undo temporary changes in all-helpers.sh
…es-development Fix submodule error messages
…doc_9632 Remove discussions of MBEDTLS_USE_PSA_CRYPTO in standalone documentation
Add a SBOM file in CycloneDX format
This commit updates the framework submodule to include commits relevant to running TF-PSA-Crypto using CMake build system. Signed-off-by: Harry Ramsey <[email protected]>
This commit updates TF-PSA-Crypto to enable out of source builds by default. Signed-off-by: Harry Ramsey <[email protected]>
The mbedtls PRs have a |
Remove Irrelevant Components for 4.0
…e-development TF-PSA-Crypto out of source build
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One suggestion, otherwise as already mentioned you probably need to rebase on top of main head.
Added debug print in tls13 ssl_tls13_write_key_share_ext
The following files are added (imported) from the main Mbed TLS repo: scripts/assemble_changelog.py scripts/check-doxy-blocks.pl scripts/check-python-files.sh scripts/doxygen.sh scripts/apidoc_full.sh scripts/recursion.pl Signed-off-by: Valerio Setti <[email protected]>
Signed-off-by: Valerio Setti <[email protected]>
Signed-off-by: Valerio Setti <[email protected]>
322163b
to
ee3cbf5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Resolves #69.
development
PR: Mbed-TLS/mbedtls#9825mbedtls-3.6
PR: Mbed-TLS/mbedtls#9826Note to self: files were moved using this command: